Skip to content

Update docs #6251

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
anna-y-perion wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update docs #6251

anna-y-perion wants to merge 5 commits into from

Conversation

@anna-y-perion
Copy link
Contributor

@anna-y-perion anna-y-perion commented Sep 10, 2025

🏷 Type of documentation

  • new bid adapter
  • update bid adapter
  • new feature
  • text edit only (wording, typos)
  • bugfix (code examples)
  • new examples

📋 Checklist

  • Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
  • For new adapters check submitting your adapter docs

@netlify
Copy link

netlify bot commented Sep 10, 2025 •
edited
Loading

✅ Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 46eb12d
🔍 Latest deploy log https://app.netlify.com/projects/prebid-docs-preview/deploys/68f9d7eb93b0b90008def9fa
😎 Deploy Preview https://deploy-preview-6251--prebid-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@anna-y-perion
Copy link
Contributor Author

anna-y-perion commented Sep 11, 2025

For convenience:
programmaticx: prebid/prebid-server#4456
tagoras: prebid/prebid-server#4329
opamarketplace: prebid/prebid-server#4526

@anna-y-perion
Copy link
Contributor Author

anna-y-perion commented Oct 15, 2025

Hi @ChrisHuie
Can you tell when it is the planed to merge these?

bsardo
bsardo reviewed Oct 16, 2025
View reviewed changes
dev-docs/bidders/programmaticX.md
Copy link

@bsardo bsardo Oct 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anna-y-perion the bidder params listed here will not be accurate after the upcoming 4.0 release as these appear to be bidder params for the ProgX adapter owned by TeqBlaze. It is my understanding from looking at the two PBS-Go PRs (prebid/prebid-server#4352 and prebid/prebid-server#4456) that ProgrammaticX will instead be an alias of vidazoo which means the lone bidder param will be cId.

Copy link
Contributor Author

@anna-y-perion anna-y-perion Oct 19, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bsardo
there is a PR that fixed it https://github.com/prebid/prebid.github.io/pull/6228/files
I am struggling to understand how to synchronise all changes around one adapter in 3 repos so all changes are visible at the same time. There are changes that come but my previous not yet merged so they look like something that is missing.

Copy link
Contributor Author

@anna-y-perion anna-y-perion Oct 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bsardo FYI the https://github.com/prebid/prebid.github.io/pull/6228/files pr was merged and now it is aligned.

@bsardo bsardo mentioned this pull request Oct 16, 2025
Open
@anna-y-perion
Copy link
Contributor Author

anna-y-perion commented Oct 29, 2025 •
edited
Loading

@ChrisHuie hi,
I see this pull request is on hold. What is missing here in order to move it forward to approval? The concern Brian Sardo had been resolved. Is there anything else here?

@anna-y-perion
Copy link
Contributor Author

anna-y-perion commented Nov 16, 2025 •
edited
Loading

@ChrisHuie hi,
I would like to understand if there is issue that stops this PR of being approved or anything that puts it in "on hold" label?
I really would like to push it forward.
Thank you in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bsardo bsardo bsardo left review comments

@ChrisHuie ChrisHuie ChrisHuie approved these changes

Assignees

No one assigned

Labels

LGTM not merged yet Pending Next Major Release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@anna-y-perion @bsardo @ChrisHuie